-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
initial design for automation chain expansion and FG/LN feeds removal #11139
Conversation
I see that you haven't updated any CHANGELOG files. Would it make sense to do so? |
Go solidity wrappers are out-of-date, regenerate them via the |
Go solidity wrappers are out-of-date, regenerate them via the |
@@ -361,6 +369,12 @@ func (r *EvmRegistry) simulatePerformUpkeeps(ctx context.Context, checkResults [ | |||
r.lggr.Warnf("upkeepId %s is not eligible after simulation of perform", checkResults[idx].UpkeepID.String()) | |||
checkResults[performToKeyIdx[i]].Eligible = false | |||
checkResults[performToKeyIdx[i]].IneligibilityReason = uint8(encoding.UpkeepFailureReasonSimulationFailed) | |||
} else { | |||
// at this point, the core node knows the exact perform data of the upkeep and the call data to L1. | |||
// it can calculate a relatively accurate L1 gas cost |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this may introduce some delays between when L1 gas cost is calculated and when users are charged.
but currently the data feeds are updated every 15 mins or the deviation is above x %
Go solidity wrappers are out-of-date, regenerate them via the |
@@ -190,6 +190,14 @@ func (r *EvmRegistry) checkUpkeeps(ctx context.Context, payloads []ocr2keepers.U | |||
continue | |||
} | |||
|
|||
// call gas estimator (GE) component to get L2 gas cost | |||
// estimatedL1GasCost = GE.getL1GasCost(chain_id, block_number, block_hash, estimated_tx_call_data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
currently, core node does not know about maxPerformDataSize
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ryan: can we estimate with more accurate call data size?
Go solidity wrappers are out-of-date, regenerate them via the |
Go solidity wrappers are out-of-date, regenerate them via the |
Go solidity wrappers are out-of-date, regenerate them via the |
Go solidity wrappers are out-of-date, regenerate them via the |
Go solidity wrappers are out-of-date, regenerate them via the |
Go solidity wrappers are out-of-date, regenerate them via the |
SonarQube Quality Gate Reliability Rating on New Code (is worse than A) See analysis details on SonarQube Fix issues before they fail your Quality Gate with SonarLint in your IDE. |
Go solidity wrappers are out-of-date, regenerate them via the |
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
No description provided.